Skip to content

Report file not found errors (fixes issue #8).#16

Merged
robertos merged 2 commits into
microsoft:masterfrom
erikdahlstrom:throw_on_filenotfound
Mar 20, 2018
Merged

Report file not found errors (fixes issue #8).#16
robertos merged 2 commits into
microsoft:masterfrom
erikdahlstrom:throw_on_filenotfound

Conversation

@erikdahlstrom

Copy link
Copy Markdown
Contributor

Report if input files are missing.

if (!std::experimental::filesystem::exists(inputFilePath))
{
throw std::invalid_argument("Input file not found.");
}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: new line after brace

@robertos robertos left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved with a nit. Feel free to fix it or merge if you deem too little to fix :) Thanks!

@robertos robertos merged commit 172da90 into microsoft:master Mar 20, 2018
@erikdahlstrom erikdahlstrom deleted the throw_on_filenotfound branch April 4, 2018 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants